Skip to content

refactor: test: add recv_channel_ready_instructions helper - #213

Merged
morehouse merged 1 commit into
lnfuzz:masterfrom
ekzyis:refactor-recv-channel-ready-instructions-helper
Sep 2, 2026
Merged

refactor: test: add recv_channel_ready_instructions helper#213
morehouse merged 1 commit into
lnfuzz:masterfrom
ekzyis:refactor-recv-channel-ready-instructions-helper

Conversation

@ekzyis

@ekzyis ekzyis commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

I introduced this refactor in #163, but I decided now to cherry-pick it as a separate PR to have less commits to rebase in #163 if there are more changes to #186 or #192.

@erickcestari erickcestari left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@NishantBansal2003 NishantBansal2003 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@morehouse morehouse left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morehouse

Copy link
Copy Markdown
Collaborator

Needs rebase

@ekzyis
ekzyis force-pushed the refactor-recv-channel-ready-instructions-helper branch from ff32024 to 8baadbd Compare September 2, 2026 16:16
@ekzyis

ekzyis commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

8baadbd: rebased on master (d9abddb)

There was a conflict with 2f2a902. I resolved it by only refactoring in two places instead of three.

@morehouse
morehouse merged commit ac9ec28 into lnfuzz:master Sep 2, 2026
5 checks passed
@ekzyis
ekzyis deleted the refactor-recv-channel-ready-instructions-helper branch September 2, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants